Cleanup noisy test output#20015
Conversation
Confidence Score: 5/5Safe to merge — all changes are scoped to test infrastructure and dev-time package resolution, with no impact on published output. The No files require special attention. Reviews (5): Last reviewed commit: "use source files instead of dist files i..." | Re-trigger Greptile |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughThe pull request modifies the Comment |
193ff92 to
71bb1a4
Compare
In other packages we use the src files as well and only use the dist/ files when we actually publish. This solves an issue where we _have_ to build `@tailwindcss/node` first, and gets rid of some of the warnings you get when running tests.
71bb1a4 to
1cb7254
Compare
This PR cleans up the noisy test output where some
console.warnmessages were leaking into the test output.This also uses the
src/files instead of thedist/files of@tailwindcss/nodeto get rid of a source map related warning in tests. It also means that we don't have to rebuild@tailwindcss/nodewhen we make changes.Test plan
vitest run --reporter=minimal)Before:

After:
